Skip to content

Remove IDE version numbers from the editors name #2029

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
azatsarynnyy merged 1 commit into from
Aug 18, 2025
Merged

Remove IDE version numbers from the editors name #2029

azatsarynnyy merged 1 commit into from
Aug 18, 2025

Conversation

@azatsarynnyy
Copy link
Member

@azatsarynnyy azatsarynnyy commented Aug 15, 2025 •
edited
Loading

What does this PR do?

With https://issues.redhat.com/browse/CRW-8933 implemented, it's possible for the admin to propagate any JetBrains IDE version that will be deployed to the users' workspaces.
So, it doesn't make sense to display the fixed version on the Dashboard:

Before:
image

After:
image

Screenshot/screencast of this PR

What issues does this PR fix or reference?

How to test this PR?

  1. Prepare a patch file if needed:
cat > /tmp/cr-patch.yaml <<EOF
apiVersion: org.eclipse.che/v2
kind: CheCluster
spec: {}
EOF
  1. Deploy the operator:

OpenShift

./build/scripts/olm/test-catalog-from-sources.sh --cr-patch-yaml /tmp/cr-patch.yaml

on Minikube

./build/scripts/minikube-tests/test-operator-from-sources.sh --cr-patch-yaml /tmp/cr-patch.yaml

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

@tolusha
Copy link
Contributor

tolusha commented Aug 18, 2025

/retest

@openshift-ci openshift-ci bot added the lgtm label Aug 18, 2025
@openshift-ci
Copy link

openshift-ci bot commented Aug 18, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: azatsarynnyy, tolusha

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tolusha
Copy link
Contributor

tolusha commented Aug 18, 2025

/retest

@azatsarynnyy azatsarynnyy merged commit fc185b7 into main Aug 18, 2025
35 of 38 checks passed
@azatsarynnyy azatsarynnyy deleted the upd-editor-defs branch August 18, 2025 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tolusha tolusha tolusha approved these changes

@SDawley SDawley Awaiting requested review from SDawley SDawley is a code owner

@ibuziuk ibuziuk Awaiting requested review from ibuziuk ibuziuk is a code owner

@mkuznyetsov mkuznyetsov Awaiting requested review from mkuznyetsov mkuznyetsov is a code owner

@dkwon17 dkwon17 Awaiting requested review from dkwon17 dkwon17 is a code owner

Assignees

@tolusha tolusha

Labels

lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@azatsarynnyy @tolusha